Skip to content

Add docs link#8977

Merged
amyblais merged 1 commit into
amyblais-fix101116from
amyblais-docslinks
May 18, 2026
Merged

Add docs link#8977
amyblais merged 1 commit into
amyblais-fix101116from
amyblais-docslinks

Conversation

@amyblais
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 18, 2026 06:29
@amyblais amyblais added the 2: Editor Review Requires review by an editor label May 18, 2026
@amyblais amyblais requested a review from Combs7th May 18, 2026 06:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a link to the mmctl command-line tool documentation from a bullet in the Mattermost v11 changelog entry describing the new --workers flag.

Changes:

  • Converts plain text "mmctl import process" into a hyperlink in the v11.7 changelog section.

Comment thread source/product-overview/mattermost-v11-changelog.md
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

The PR updates a single line in the v11.7 changelog to add a documentation link for the --workers flag used in the mmctl import process, with adjusted inline code formatting to improve readability and cross-reference clarity.

Changes

v11.7 Changelog Documentation Update

Layer / File(s) Summary
mmctl --workers documentation link update
source/product-overview/mattermost-v11-changelog.md
The --workers flag bullet in the v11.7 Administration improvements section now links to the mmctl import documentation, with adjusted inline code and link formatting.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • mattermost/docs#8975: Both PRs update documentation links in the same v11.7 mattermost-v11-changelog.md section for the Administration improvements subsection.

Suggested labels

2: Editor Review

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Add docs link' is vague and does not clearly indicate what documentation link is being added or to which section. Consider a more specific title such as 'Add mmctl documentation link to v11.7 changelog' to clarify what docs link is being added.
Description check ❓ Inconclusive No pull request description was provided by the author, so it is impossible to assess the intent or context of the changes. Add a brief description explaining why the documentation link is being added and what it references.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch amyblais-docslinks

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@source/product-overview/mattermost-v11-changelog.md`:
- Line 88: Replace the hardcoded absolute docs URL in the changelog line that
references the mmctl import process (the "[mmctl import process]" link
mentioning the "--workers" flag) with the repo-appropriate relative Markdown
link to the corresponding docs page (use the relative path equivalent of
/administration-guide/manage/mmctl-command-line-tool.html), and apply the same
relative-link style to other internal Mattermost docs links in this file for
consistency.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 73b679b1-c4c4-4766-9774-74ae689920dc

📥 Commits

Reviewing files that changed from the base of the PR and between 6140523 and 7b4fe46.

📒 Files selected for processing (1)
  • source/product-overview/mattermost-v11-changelog.md

Comment thread source/product-overview/mattermost-v11-changelog.md
@github-actions
Copy link
Copy Markdown
Contributor

Newest code from mattermost has been published to preview environment for Git SHA 7b4fe46

@amyblais amyblais changed the base branch from master to amyblais-fix101116 May 18, 2026 15:31
@amyblais amyblais removed the 2: Editor Review Requires review by an editor label May 18, 2026
@amyblais amyblais merged commit 1adaecc into amyblais-fix101116 May 18, 2026
13 checks passed
@amyblais amyblais deleted the amyblais-docslinks branch May 18, 2026 15:31
amyblais added a commit that referenced this pull request May 18, 2026
* Update mattermost-v10-changelog.md

* Update mattermost-v10-changelog.md

* Update mattermost-v10-changelog.md

* Update mattermost-v10-changelog.md

* Add docs link (#8977)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants